From: Szabolcs Hubai Date: Sun, 7 Dec 2025 20:04:13 +0000 (+0100) Subject: snapcast: define ALSA support explicitly X-Git-Url: http://git.openwrt.org/%22http:/oss.oetiker.ch/rrdtool//%22/%22http:/oss.oetiker.ch/rrdtool/%22?a=commitdiff_plain;h=a77e5889bc43e4d6589a6f4784b2f9a4488a8839;p=feed%2Fpackages.git snapcast: define ALSA support explicitly As Hannu Nyman pointed out, in some cases the build of the package is able to fail: Package snapserver is missing dependencies for the following libraries: libasound.so.2 He said: on some build runs, alsa is built (as required by other packages), and if it is built before snapcast, it may be detected, if the feature is not explicitly disabled in CMakeLists.txt. Also: "I think that you could drop that AUDIO_SUPPORT conditional. AUDIO_SUPPORT is selected e.g. if USB_SUPPORT is selected, so it is not that big restriction." Suggested-by: Hannu Nyman Signed-off-by: Szabolcs Hubai --- diff --git a/sound/snapcast/Makefile b/sound/snapcast/Makefile index b097bad2f5..20b57b7f61 100644 --- a/sound/snapcast/Makefile +++ b/sound/snapcast/Makefile @@ -21,6 +21,7 @@ include $(INCLUDE_DIR)/cmake.mk CMAKE_OPTIONS += \ -DBUILD_TESTS=OFF \ + -DBUILD_WITH_ALSA=ON \ -DBUILD_WITH_AVAHI=OFF \ -DBUILD_WITH_PULSE=OFF @@ -28,7 +29,7 @@ define Package/snapcast/Default SECTION:=sound CATEGORY:=Sound TITLE:=Synchronous multiroom audio player - DEPENDS:=+AUDIO_SUPPORT:alsa-lib +libexpat +libatomic +libogg +libflac +libopus +boost +libsoxr + DEPENDS:=+alsa-lib +libexpat +libatomic +libogg +libflac +libopus +boost +libsoxr URL:=https://github.com/badaix/snapcast endef